Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove libatomic_ops dependency #143

Merged
merged 7 commits into from
Dec 8, 2021

Conversation

straight-shoota
Copy link
Member

libatomic_ops is no longer required for building bdwgc starting with 8.0.0. We're using newer versions now, so there's no need to pull in libatomic_ops.

libatomic_ops is no longer required for building bdwgc starting with
8.0.0. We're using newer versions now, so there's no need to pull
in libatomic_ops.
@straight-shoota
Copy link
Member Author

Copy link
Member

@beta-ziliani beta-ziliani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 it feels so good to remove a dependency!

@straight-shoota
Copy link
Member Author

straight-shoota commented Oct 4, 2021

CI shows a lot of failures. Most of them are unrelated to this PR (crystal-lang/crystal#11275, and the known issues with i386 and alpine).

I don't know what's going on with the dist_linux job: https://app.circleci.com/pipelines/github/crystal-lang/crystal/6937/workflows/721b58ad-2c44-432e-998f-580b2c579f57/jobs/64226

Step 7/17 : RUN curl https://ftp.pcre.org/pub/pcre/pcre-${libpcre_version}.tar.gz | tar -zx  && cd pcre-${libpcre_version}  && ./configure --disable-shared --disable-cpp --enable-jit --enable-utf --enable-unicode-properties  && make -j$(nproc)
 ---> Running in 0526008b18b2
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0100   162  100   162    0     0    285      0 --:--:-- --:--:-- --:--:--   285

gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
The command '/bin/sh -c curl https://ftp.pcre.org/pub/pcre/pcre-${libpcre_version}.tar.gz | tar -zx  && cd pcre-${libpcre_version}  && ./configure --disable-shared --disable-cpp --enable-jit --enable-utf --enable-unicode-properties  && make -j$(nproc)' returned a non-zero code: 2
make: *** [build/bundled-libs.tar] Error 2

Looks like the download is failing. I can't reproduce that locally. But it appears to be consistens in CI.

@straight-shoota
Copy link
Member Author

@straight-shoota
Copy link
Member Author

There were a couple of failures in CI, but none seem to be directly related to this change. The obvious i386 failures can be entirely ignored.

@beta-ziliani
Copy link
Member

maybe the name of the branch is a problem for macOS?

The `version' component of Mac package names can only include alphabetical characters (a-z, A-Z), numbers (0-9), dots  (.), plus signs (+), and dashes (-). Converting `ci-remove-libatomic_ops-dev' to `ci-remove-libatomic-ops-dev

@bcardiff
Copy link
Member

bcardiff commented Oct 7, 2021

I couldn't find documentation about whether or not _ is allowed, but the git branch name is used to determine the snap branch in https://github.com/crystal-lang/crystal/blob/master/.circleci/config.yml#L237-L238 there we replace /, we can do the same to allow branches with _.

@straight-shoota
Copy link
Member Author

I added a change to fix the branch name sanitation: crystal-lang/crystal@ca0db73

Now the previously failing CI jobs succeed: https://app.circleci.com/pipelines/github/crystal-lang/crystal/6981/workflows/94c27061-f6c0-4e12-b58b-252c9a03c39d

Thanks for the pointers 👍

@straight-shoota
Copy link
Member Author

@straight-shoota
Copy link
Member Author

@straight-shoota straight-shoota merged commit 8328ae5 into master Dec 8, 2021
@straight-shoota straight-shoota deleted the refactor/remove-libatomic_ops branch December 8, 2021 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants